I've added a commit adding the BUILDs for the legacy-cli where all the compiler errors are and added the compilation of these to circleci. With all the changes landing keeping this PR up to date with the fixes without the a bazel build (#23074) is quite the hassle. Adding the actual bazel build to CI ensures this PR is up to date and correctly fixes the compilation errors.

While the bazel build of legacy-cli is unnecessary right now (other then for type checking) I think it's worth avoiding compilation errors. When these tests are moved to bazel (#23074) this will be required anyway, which hopefully will be finished soon 🤞

I've reverted the addition of webdriver-manager to the package.json since #23245 switches to the one within protractor anyway (node_modules/protractor/bin/webdriver-manager). Since it's only used in a require.resolve it's a runtime dependency anyway and isn't required to compile.

I can maybe do linting next but really need type checking before anything else 👍

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}